-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove argocd-cm plugin support #13755
chore: remove argocd-cm plugin support #13755
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #13755 +/- ##
==========================================
- Coverage 49.64% 49.61% -0.03%
==========================================
Files 256 256
Lines 43914 43800 -114
==========================================
- Hits 21800 21733 -67
+ Misses 19958 19934 -24
+ Partials 2156 2133 -23
☔ View full report in Codecov by Sentry. |
I worry about this one because it's such a widely used feature. Could be perhaps just add a deprecation warning for a few versions before removing? |
@todaywasawesome it's been deprecated since 2.5, and we've pushed removal twice. 😆 https://argo-cd.readthedocs.io/en/latest/operator-manual/upgrading/2.4-2.5/#argocd-cm-plugins-cmps-are-deprecated |
You can always count on me to ask the critical questions 🥴 |
It's a fair point though. I fully expect people will be on this thread two years from now saying "aw man, why did y'all do this, I'm just upgrading from 1.9, this sucks." :-P |
@crenshaw-dev could you please fix conflicts? |
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
b07f72b
to
6216c28
Compare
@pasha-codefresh this is ready for another look |
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@pasha-codefresh wanna glance at the e2e test fixes? |
Thank you |
Fixes #8117